feat(nitro): Handle sourcemap preparation and upload by logaretm · Pull Request #19304 · getsentry/sentry-javascript

logaretm · 2026-02-12T17:37:29Z

Adds automatic sourcemap handling to the Nitro SDK, using @sentry/bundler-plugin-core for builder-agnostic post-build upload.

Nitro uses rollup or rolldown, so it made sense to make it as agnostic as possible.

Closes #17992

This PR is part of a stack:

linear · 2026-02-12T17:37:32Z

JS-1076 [Nitro] Upload sourcemaps

github-actions · 2026-02-12T17:48:43Z

Codecov Results 📊

Generated by Codecov Action

github-actions · 2026-02-12T17:49:14Z

size-limit report 📦

Path	Size	% Change	Change
@sentry/browser	25.88 kB	added	added
@sentry/browser - with treeshaking flags	24.35 kB	added	added
@sentry/browser (incl. Tracing)	43.77 kB	added	added
@sentry/browser (incl. Tracing + Span Streaming)	45.47 kB	added	added
@sentry/browser (incl. Tracing, Profiling)	48.7 kB	added	added
@sentry/browser (incl. Tracing, Replay)	82.89 kB	added	added
@sentry/browser (incl. Tracing, Replay) - with treeshaking flags	72.4 kB	added	added
@sentry/browser (incl. Tracing, Replay with Canvas)	87.58 kB	added	added
@sentry/browser (incl. Tracing, Replay, Feedback)	99.83 kB	added	added
@sentry/browser (incl. Feedback)	42.7 kB	added	added
@sentry/browser (incl. sendFeedback)	30.55 kB	added	added
@sentry/browser (incl. FeedbackAsync)	35.55 kB	added	added
@sentry/browser (incl. Metrics)	27.16 kB	added	added
@sentry/browser (incl. Logs)	27.29 kB	added	added
@sentry/browser (incl. Metrics & Logs)	27.98 kB	added	added
@sentry/react	27.62 kB	added	added
@sentry/react (incl. Tracing)	46.01 kB	added	added
@sentry/vue	30.7 kB	added	added
@sentry/vue (incl. Tracing)	45.58 kB	added	added
@sentry/svelte	25.89 kB	added	added
CDN Bundle	28.55 kB	added	added
CDN Bundle (incl. Tracing)	44.82 kB	added	added
CDN Bundle (incl. Logs, Metrics)	29.93 kB	added	added
CDN Bundle (incl. Tracing, Logs, Metrics)	45.91 kB	added	added
CDN Bundle (incl. Replay, Logs, Metrics)	68.83 kB	added	added
CDN Bundle (incl. Tracing, Replay)	81.78 kB	added	added
CDN Bundle (incl. Tracing, Replay, Logs, Metrics)	82.85 kB	added	added
CDN Bundle (incl. Tracing, Replay, Feedback)	87.29 kB	added	added
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics)	88.36 kB	added	added
CDN Bundle - uncompressed	83.4 kB	added	added
CDN Bundle (incl. Tracing) - uncompressed	134.03 kB	added	added
CDN Bundle (incl. Logs, Metrics) - uncompressed	87.55 kB	added	added
CDN Bundle (incl. Tracing, Logs, Metrics) - uncompressed	137.44 kB	added	added
CDN Bundle (incl. Replay, Logs, Metrics) - uncompressed	210.91 kB	added	added
CDN Bundle (incl. Tracing, Replay) - uncompressed	251.26 kB	added	added
CDN Bundle (incl. Tracing, Replay, Logs, Metrics) - uncompressed	254.66 kB	added	added
CDN Bundle (incl. Tracing, Replay, Feedback) - uncompressed	264.18 kB	added	added
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics) - uncompressed	267.56 kB	added	added
@sentry/nextjs (client)	48.54 kB	added	added
@sentry/sveltekit (client)	44.18 kB	added	added
@sentry/node-core	57.97 kB	added	added
@sentry/node	174.81 kB	added	added
@sentry/node - without tracing	97.93 kB	added	added
@sentry/aws-serverless	115.16 kB	added	added

github-actions · 2026-03-17T14:22:53Z

This pull request has gone three weeks without activity. In another week, I will close it.

But! If you comment or otherwise update it, I will reset the clock, and if you apply the label PR: no-auto-close I will leave it alone ... forever!

Copilot

Pull request overview

Adds build-time sourcemap generation + post-build sourcemap (and debug ID) handling to the @sentry/nitro SDK by wiring Nitro’s compiled hook to @sentry/bundler-plugin-core.

Changes:

Introduces packages/nitro/src/sourceMaps.ts to configure Nitro sourcemap settings and to run a post-build upload flow via createSentryBuildPluginManager.
Threads SentryNitroOptions through module setup so sourcemap behavior can be configured from withSentryConfig/setupSentryNitroModule.
Adds Vitest coverage for option-shaping and basic hook registration behavior.

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 6 comments.

Show a summary per file

File	Description
packages/nitro/src/sourceMaps.ts	New sourcemap config + post-build hook which injects debug IDs and uploads sourcemaps via bundler-plugin-core
packages/nitro/test/sourceMaps.test.ts	New unit tests for plugin option building, config mutation, and hook registration
packages/nitro/src/module.ts	Passes Sentry options into module setup to enable sourcemap handling
packages/nitro/src/config.ts	Defines `SentryNitroOptions` from bundler-plugin-core options and enables sourcemap config during module setup
packages/nitro/rollup.npm.config.mjs	Marks `@sentry/bundler-plugin-core` as external in the package build
packages/nitro/package.json	Adds `@sentry/bundler-plugin-core` dependency

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Implements HTTP server instrumentation for both `h3` and `srvx` by listening to their tracing channel events. - `h3` TC PR: h3js/h3#1251 - `srvx` TC PR: h3js/srvx#141 Closes #18123 --- **This PR is part of a stack:** - #20358 - #19224 - #19225 👈 - #19304 --------- Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

cursor

Cursor Bugbot has reviewed your changes and found 1 potential issue.

There are 3 total unresolved issues (including 2 from previous reviews).

^{❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.}

^{Reviewed by Cursor Bugbot for commit 25e2ec0. Configure here.}

Implements HTTP server instrumentation for both `h3` and `srvx` by listening to their tracing channel events. - `h3` TC PR: h3js/h3#1251 - `srvx` TC PR: h3js/srvx#141 Closes #18123 --- **This PR is part of a stack:** - #20358 - #19224 - #19225 👈 - #19304 --------- Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

…is set When `sourcemaps.disable` is `'disable-upload'`, the plugin should still inject debug IDs and keep `.map` files so users can upload them manually. Previously, both `injectDebugIds()` and `deleteArtifacts()` were incorrectly gated or ungated, causing debug IDs to be skipped and source maps to be deleted even in the manual-upload workflow. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Derive `SentryNitroOptions` from `BuildTimeOptionsBase` (from `@sentry/core`) instead of picking from the bundler plugin's internal `Options` type. This provides a stable public API that follows this repo's semver, rather than exposing bundler-plugin-specific fields like `_metaOptions` and top-level `disable`. Key changes: - `url` option renamed to `sentryUrl` (consistent with other SDKs) - Top-level `disable` removed (use `sourcemaps.disable` instead) - `_metaOptions` no longer user-configurable Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

…ing `true` Previously, `configureSourcemapSettings` unconditionally set `config.sourcemap = true`, overriding the user's explicit `false`. This could expose source maps publicly when users intentionally disabled them. Now follows the same 3-case pattern as other meta-framework SDKs (Nuxt): 1. User disabled (false) → keep their setting, warn about unminified errors 2. User enabled (true) → keep their setting 3. User didn't set (undefined) → enable for Sentry Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

`configureSourcemapSettings` runs at build time, but the `debug` logger from `@sentry/core` is only initialized by `Sentry.init()` at runtime. Switch to `console.warn`/`console.log` so build-time warnings (e.g. source maps explicitly disabled) are actually visible to developers. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

The `rewriteSources` option was hardcoded to `normalizePath`, silently discarding any custom function provided by the user. Now falls back to `normalizePath` only when the user doesn't provide their own function, matching the behavior of other meta-framework SDKs (Nuxt, Next.js). Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Previously `filesToDeleteAfterUpload` defaulted to `['**/*.map']` unconditionally, deleting user-generated source maps even when the user explicitly set `sourcemap: true` in their Nitro config. Now the default only applies when Sentry was the one to enable sourcemap generation. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

Nitro spawns a nested Nitro instance for prerendering with the user's `modules` array re-installed, which caused the Sentry module to run twice and upload source maps + create a release twice per build. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

s1gr1d · 2026-04-20T11:27:53Z

+      // We don't prepare the artifacts because we injected debug IDs manually before
+      prepareArtifacts: false,
+    });
+    await sentryBuildPluginManager.deleteArtifacts();


Did you manually also check if this deletes the correct source maps - based on the user-set source map setting?

I tried 3 scenarios locally:

default: Sourcemaps files get deleted

explicit true, sourcemap files is retained

disable-upload: retained but not uploaded

I did miss that it should only delete nitro's sourcemaps (fixed now in 1c13825) but not sure what other scenarios to take into account. Also checked that it respects the filesToDeleteAfterUpload option, is that what you meant?

`filesToDeleteAfterUpload` previously defaulted to `['**/*.map']`, which is resolved from the project root and could traverse `node_modules`. Scope the default to `nitro.options.output.serverDir` so we only touch maps we generated. Also log the chosen glob in debug mode to match the Nuxt SDK UX. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

sentry · 2026-04-20T15:29:52Z

+  } else {
+    // User did not explicitly set sourcemap — enable it for Sentry
+    config.sourcemap = true;
+    sentryEnabledSourcemaps = true;
+    if (moduleOptions?.debug) {


Bug: The function only checks for boolean config.sourcemap values, incorrectly handling valid string options like "inline" by falling into the default case and overwriting the user's setting.
_{Severity: MEDIUM}

Suggested Fix

Update the conditional logic to correctly handle string values for config.sourcemap. Instead of strict boolean checks, use a condition that includes valid string options, such as ['hidden', true].includes(config.sourcemap), to determine if sourcemaps are enabled. This will prevent the else block from incorrectly overwriting the user's configuration.

Prompt for AI Agent

Review the code at the location below. A potential bug has been identified by an AI agent. Verify if this is a real issue. If it is, propose a fix; if not, explain why it's not valid. Location: packages/nitro/src/sourceMaps.ts#L158-L162 Potential issue: The `configureSourcemapSettings` function checks for `config.sourcemap` using strict boolean comparisons (`=== true` and `=== false`). When a user provides a valid string value like `"inline"` or `"hidden"`, these checks fail. The code then executes the `else` block, which is intended for when the option is unset. This block incorrectly overwrites the user's setting with `config.sourcemap = true` and enables `sentryEnabledSourcemaps`. As a result, the user's intended sourcemap configuration is ignored, and the generated sourcemaps are incorrectly marked for deletion after upload.

This was referenced Feb 12, 2026

feat(nitro): Instrument HTTP Server #19225

Merged

feat(nitro): Nitro SDK #19224

Open

logaretm mentioned this pull request Feb 12, 2026

fix: preserve sourcemap mappings for chunks containing user code nitrojs/nitro#4031

Merged

8 tasks

logaretm force-pushed the awad/js-1128-nitro-use-tracing-channels-for-srvx-and-h3 branch from 1923dda to be2f037 Compare February 23, 2026 14:50

logaretm force-pushed the awad/js-1076-nitro-upload-sourcemaps branch from 0d86a79 to 9f6254a Compare February 23, 2026 14:50

logaretm force-pushed the awad/js-1128-nitro-use-tracing-channels-for-srvx-and-h3 branch from be2f037 to 3b30e36 Compare February 23, 2026 14:52

logaretm force-pushed the awad/js-1076-nitro-upload-sourcemaps branch from 9f6254a to 63a2175 Compare February 23, 2026 14:52

logaretm force-pushed the awad/js-1128-nitro-use-tracing-channels-for-srvx-and-h3 branch from 3b30e36 to 36c96b6 Compare February 23, 2026 16:17

logaretm force-pushed the awad/js-1076-nitro-upload-sourcemaps branch 2 times, most recently from d3e3ae1 to a24cad5 Compare February 23, 2026 16:23

github-actions bot added the PR: stale label Mar 17, 2026

logaretm added PR: no-auto-close and removed PR: stale labels Mar 17, 2026

logaretm force-pushed the awad/js-1128-nitro-use-tracing-channels-for-srvx-and-h3 branch 2 times, most recently from 3cf1515 to 342ea3b Compare April 15, 2026 13:54

logaretm force-pushed the awad/js-1076-nitro-upload-sourcemaps branch from a24cad5 to 1583866 Compare April 15, 2026 14:44

logaretm force-pushed the awad/js-1128-nitro-use-tracing-channels-for-srvx-and-h3 branch from 342ea3b to 8d44437 Compare April 15, 2026 14:55

logaretm force-pushed the awad/js-1076-nitro-upload-sourcemaps branch 2 times, most recently from a967e7a to 00bd835 Compare April 15, 2026 14:59

logaretm marked this pull request as ready for review April 15, 2026 18:13

logaretm requested review from andreiborza, Copilot and s1gr1d April 15, 2026 18:13

Copilot started reviewing on behalf of logaretm April 15, 2026 18:14 View session

sentry bot reviewed Apr 15, 2026

View reviewed changes

Comment thread packages/nitro/src/sourceMaps.ts

Copilot AI reviewed Apr 15, 2026

View reviewed changes

sentry bot reviewed Apr 17, 2026

View reviewed changes

Comment thread packages/nitro/src/sourceMaps.ts Outdated

logaretm force-pushed the awad/create-nitro-sdk-metadata branch from 4cd6415 to 47d85ab Compare April 17, 2026 15:54

logaretm force-pushed the awad/js-1076-nitro-upload-sourcemaps branch from d3328c8 to 25e2ec0 Compare April 17, 2026 15:55

sentry bot reviewed Apr 17, 2026

View reviewed changes

Comment thread packages/nitro/src/sourceMaps.ts Outdated

cursor bot reviewed Apr 17, 2026

View reviewed changes

Comment thread packages/nitro/src/sourceMaps.ts Outdated

sentry bot reviewed Apr 17, 2026

View reviewed changes

Comment thread packages/nitro/src/sourceMaps.ts

logaretm force-pushed the awad/create-nitro-sdk-metadata branch from 47d85ab to fa580dd Compare April 18, 2026 03:53

logaretm and others added 13 commits April 17, 2026 23:53

feat: sourcemaps upload and debug ID injection

572a830

ref: minor refactor

d3b0f0f

fix: boolean oopsie

06d6b67

test: update test expectations

5117570

fix: revert change

ea8606e

test: fix unit test assertion

d114ac3

logaretm force-pushed the awad/js-1076-nitro-upload-sourcemaps branch from ff591f3 to 14b31ea Compare April 18, 2026 03:53

sentry bot reviewed Apr 18, 2026

View reviewed changes

Comment thread packages/nitro/src/sourceMaps.ts

s1gr1d reviewed Apr 20, 2026

View reviewed changes

logaretm and others added 3 commits April 20, 2026 09:36

fix: skip sourcemap upload hooks early

52626ee

fix: ensure we only delete nitro's sourcemaps

1c13825

logaretm requested a review from s1gr1d April 20, 2026 15:25

sentry bot reviewed Apr 20, 2026

View reviewed changes

Uh oh!

Conversation

logaretm commented Feb 12, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

linear bot commented Feb 12, 2026

Uh oh!

github-actions bot commented Feb 12, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Codecov Results 📊

Uh oh!

github-actions bot commented Feb 12, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

size-limit report 📦

Uh oh!

github-actions bot commented Mar 17, 2026

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

cursor bot left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

s1gr1d Apr 20, 2026

Choose a reason for hiding this comment

Uh oh!

logaretm Apr 20, 2026

Choose a reason for hiding this comment

Uh oh!

sentry bot Apr 20, 2026

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

logaretm commented Feb 12, 2026 •

edited

Loading

github-actions bot commented Feb 12, 2026 •

edited

Loading

github-actions bot commented Feb 12, 2026 •

edited

Loading